Skip to content

Pipeline to facilitate Roslyn Copilot publish #8508

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

AbhitejJohn
Copy link
Contributor

@AbhitejJohn AbhitejJohn commented Aug 7, 2025

This pipeline ensures we get the latest roslyn copilot bits. IT needs to be approved explicitly from someone with a SAW, so I've left it as a manual trigger for now. I could imagine a release flow though that auto triggers some of these and that one person with a SAW makes all the approvals needed at once.

Pipeline is here: https://devdiv.visualstudio.com/DevDiv/_build?definitionId=27222

@AbhitejJohn AbhitejJohn requested a review from a team as a code owner August 7, 2025 21:32
trigger: none
pr: none

resources:
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Kind of a meta question here. Should this pipeline even be in this repo? Would conversations be a better place?

For example if you want to change the folder name where it finds the conversations build artifacts to upload. Now you have to make the change across two repos. If this was in conversations you just updated both at the same time.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good question, my thinking was having this closer to the component that actually needs it. The conversations repo has a slightly different set of processes tied to build and release pipelines which are tied to VS. Having it here gives us the freedom to iterate on this quicker. It also keeps this in our teams cross hairs a little more working with the processes we have - its been hard figuring out where things exist, so trying to keep some of the infra pieces atleast in fewer places.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Understandable. If we were totally disregarding process then I'd definitely prefer this lives in conversations (conversations produces the assets, so it should also own the code publishing them).

But if its going to be a significant burden to put it there, I'm not that opposed to it being here.

@AbhitejJohn
Copy link
Contributor Author

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@AbhitejJohn
Copy link
Contributor Author

@dibarbet : Is there an infra problem causing these test failures? I wouldn't expect these changes to fail the build. I could also use another review on this as well so I can close this out :)

@dibarbet
Copy link
Member

Is there an infra problem causing these test failures? I wouldn't expect these changes to fail the build. I could also use another review on this as well so I can close this out :)

Yeah looks like there are some new flaky tests

@dibarbet
Copy link
Member

Have one question here - #8508 (comment)

@AbhitejJohn
Copy link
Contributor Author

Have one question here - #8508 (comment)

And responded : #8508 (comment) :) I'm happy to chat if you feel strongly though.

@AbhitejJohn
Copy link
Contributor Author

/azp run dotnet.vscode-csharp

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@dibarbet
Copy link
Member

/azp run dotnet.vscode-csharp

you should be able to just re-queue the individual job (either from the github checks page or AzDo page

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants